Skip to content

Conversation

nordicjm
Copy link
Contributor

@nordicjm nordicjm commented Oct 6, 2025

Fixes an issue caused by using a chosen node string as a node rather than looking the node up first so it could be used

@zephyrbot zephyrbot added size: XS A PR changing only a single line of code area: MCUBoot labels Oct 6, 2025
@zephyrbot zephyrbot requested review from d3zd3z and de-nordic October 6, 2025 10:46
@nordicjm nordicjm requested review from pdgendt and gchwier October 6, 2025 10:49
@nordicjm nordicjm added the Regression Something, which was working, does not anymore label Oct 6, 2025
pdgendt
pdgendt previously approved these changes Oct 6, 2025
Fixes an issue caused by using a chosen node string as a node
rather than looking the node up first so it could be used

Signed-off-by: Jamie McCrae <[email protected]>
Copy link

sonarqubecloud bot commented Oct 6, 2025

Copy link
Contributor

@gchwier gchwier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixes issue with upgrading nrf54l15dk,
tested with command:
./scripts/twister -vv -ll debug -T tests/boot/with_mcumgr --west-flash=--recover -p nrf54l15dk/nrf54l15/cpuapp --device-serial /dev/ttyACM1 --device-testing --enable-slow -s boot.with_mcumgr.test_upgrade --pytest-args='-k test_upgrade_with_revert' -c

@butok butok self-requested a review October 6, 2025 14:13
@de-nordic de-nordic added this to the v4.3.0 milestone Oct 7, 2025
@JarmouniA JarmouniA added the Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. label Oct 7, 2025
@JarmouniA
Copy link
Contributor

Blocking CI
#95630 (comment)
#96998 (comment)

@jhedberg jhedberg merged commit 2a1444e into zephyrproject-rtos:main Oct 7, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: MCUBoot Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. Regression Something, which was working, does not anymore size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants